-
Notifications
You must be signed in to change notification settings - Fork 225
Hold table locks when needed during analyze #5060
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 8.0
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 8/569 tests failed ⚠.
The first 10 failing tests are:
sc_truncate
phys_rep_perf
sc_timepart
sc_timepart_logicalsc_generated
analyze_partial_index_off_generated
analyze
incremental_backup_load
truncatesc_offline_generated
e08bc34
to
9bb6316
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 364/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart
yast
yast_stat4scan_generated
analyze_recover_deadlock
analyze_partial_index_off_generated
analyze
autoanalyze
temp_table_truncate_leak
sc_resume2
3320810
to
1e9a0ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 4/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart_logicalsc_generated
incremental_backup_load
truncatesc_offline_generated
5fc9635
to
45909ef
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 7/569 tests failed ⚠.
The first 10 failing tests are:
sc_truncate
sc_transactional_rowlocks_generated
phys_rep_perf
sc_timepart
rem
incremental_backup_load
truncatesc_offline_generated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 4/569 tests failed ⚠.
The first 10 failing tests are:
logfill_logput_window_generated
phys_rep_perf
incremental_backup_load
truncatesc_offline_generated
3016acf
to
7106d75
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart_logicalsc_generated
analyze_partial_index_off_generated
analyze
incremental_backup_load
truncatesc_offline_generated
7106d75
to
fde1743
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart
sc_timepart_logicalsc_generated
nightmare_recovery
incremental_backup_load
truncatesc_offline_generated
fde1743
to
49f2f4f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 8/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf
sc_timepart
analyze_partial_index_off_generated
sc_resume2
sc_downgrade
incremental_backup_load
truncatesc_offline_generated
49f2f4f
to
9e005f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 5/569 tests failed ⚠.
The first 10 failing tests are:
phys_rep_perf_revconn_generated
sc_timepart
sc_resume2
incremental_backup_load
truncatesc_offline_generated
Signed-off-by: Morgan Douglas <[email protected]>
9e005f7
to
aea615b
Compare
The changes in this PR guard
dbtable
access with a table lock